Skip to content

PR Project Board Label Updates#2902

Closed
gspetro-NOAA wants to merge 105 commits intoufs-community:developfrom
gspetro-NOAA:feature/add-labels
Closed

PR Project Board Label Updates#2902
gspetro-NOAA wants to merge 105 commits intoufs-community:developfrom
gspetro-NOAA:feature/add-labels

Conversation

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

@gspetro-NOAA gspetro-NOAA commented Sep 23, 2025

Commit Queue Requirements:

  • This PR addresses a relevant WM issue (if not, create an issue).
  • All subcomponent pull requests (if any) have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines), preferably on Ursa (Derecho or Hercules are acceptable alternatives). Exceptions: documentation-only PRs, CI-only PRs, etc.
    • Commit log file w/full results from RT suite run (if applicable). (N/A)
    • Commit 'test_changes.list' (unless file is empty).
  • Fill out all sections of this template.

Description:

This PR adds functionality to adjust the PR project board status automatically based on labels that are added or removed. Specifically, it handles labels for baseline updates and input data. I also included a small update to the PR template, which @DeniseWorthen requested.

I tested the auto-labeling functionality using PRs from my fork (feature/add-labels branch) to the NOAA-EPIC/ufs-weather-model (feature/pr-management branch). For example, see: NOAA-EPIC WM PR#12 and the NOAA-EPIC project board.

For security reasons, these GitHub actions won't run in a fork, so this functionality will only work once added to the WM repo. Since I have switched the code to point to the ufs-community project board, it will not work until merged. However, PR #13 (where I am working on logic for the priority labels) and the NOAA-EPIC project board show how it would work for anyone interested in taking a look.

The details around security issues and testing are basically the same as those in PR #2908. Check out that PR description for more details.

A later PR will automate priority labels, but that logic is still in the works.

Commit Message:

* UFSWM - Automate updating baseline and input data status for PR project board

Priority:

  • Critical Bugfix: Reason
  • High: Reason
  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

  • None

UFSWM Blocking Dependencies:

  • Blocked by #
  • None

Documentation:

  • Documentation update required.
    • Relevant updates are included with this PR.
    • A WM issue has been opened to track the need for a documentation update; a person responsible for submitting the update has been assigned to the issue (link issue).
  • Documentation update NOT required.
    • Explanation: This functionality does not apply to general users.

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Adds New Tests/Baselines.
  • PR Updates/Changes Baselines.
  • No Baseline Changes.

Input data Changes:

  • None.
  • New input data.
  • Updated input data.

Library Changes/Upgrades:

  • Required
    • Library names w/versions:
    • Git Stack Issue (JCSDA/spack-stack#)
  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • GaeaC6
    • Derecho
    • Ursa
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@gspetro-NOAA gspetro-NOAA changed the title Feature/add labels PR Project Board Label Updates Sep 23, 2025
@gspetro-NOAA gspetro-NOAA added the No Baseline Change No Baseline Change label Sep 24, 2025
@gspetro-NOAA gspetro-NOAA reopened this Sep 24, 2025
@gspetro-NOAA gspetro-NOAA marked this pull request as ready for review September 29, 2025 13:36
@gspetro-NOAA gspetro-NOAA moved this from Evaluating to Review in PRs to Process Sep 29, 2025
@gspetro-NOAA gspetro-NOAA moved this from Review to Schedule in PRs to Process Sep 29, 2025
<!--
Provide a list of subcomponents involved with this PR and include links to subcomponent PRs.
Example:
* FV3: NOAA-EMC/fv3atm#734
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to also edit the leftover FV3 references in this section to UFSATM when I add it to #2913 if that's ok with you.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! Works for me :)

dpsarmie added a commit to dpsarmie/ufs-weather-model that referenced this pull request Oct 1, 2025
jkbk2004 pushed a commit that referenced this pull request Oct 2, 2025
…d Label Updates #2902 (#2913)

* UFSWM - Fix Atmos%iau_offset bug, remove goto statements from noahmp, add spack ci, and add changes to PR proj board
  * UFSATM - Set Atmos%iau_offset to iau_offset in module_fcst_grid_comp
  * NOAHMP - Remove goto statements and remove leftover debug write statement

---------

Co-authored-by: gspetro-NOAA <gillian.petro@noaa.gov>
Co-authored-by: Alexander Richert <alexander.richert@noaa.gov>
@github-project-automation github-project-automation bot moved this from Schedule to Done in PRs to Process Oct 2, 2025
@gspetro-NOAA
Copy link
Copy Markdown
Collaborator Author

Merged in w/#2913.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Baseline Change No Baseline Change

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants